auto-format by kevinbackhouse · Pull Request #137 · GitHubSecurityLab/seclab-taskflow-agent

kevinbackhouse · 2026-01-26T12:33:07Z

I ran the auto-formatting tool:

hatch fmt

-#from mcp.server.fastmcp import FastMCP, Context
-from fastmcp import FastMCP, Context # use FastMCP 2.0
+
+# from mcp.server.fastmcp import FastMCP, Context


+    def flush(self) -> None: ...


 class Spec(object):


+    def write(self, b: str) -> int: ...

-    def flush(self) -> None:
-        ...


+    def closed(self) -> bool: ...

-    def write(self, b: str) -> int:
-        ...


+    def fileno(self) -> int: ...

    @property
-    def closed(self) -> bool:


+from agents import (
+    Agent,
+    Runner,
+    AgentHooks,
+    RunHooks,
+    result,
+    function_tool,
+    Tool,
+    RunContextWrapper,
+    TContext,
+    OpenAIChatCompletionsModel,
+    set_default_openai_client,
+    set_default_openai_api,
+    set_tracing_disabled,
+)


Copilot

Pull request overview

This pull request applies auto-formatting to the codebase using hatch fmt. The changes are entirely stylistic and do not alter the functionality of the code.

Changes:

Converted single quotes to double quotes throughout the codebase for string consistency
Reformatted multi-line function calls and definitions for better readability
Added/removed blank lines to comply with PEP 8 spacing guidelines (e.g., two blank lines between top-level definitions)
Adjusted line wrapping for long lines
Removed trailing whitespace and extra blank lines at the end of files

Reviewed changes

Copilot reviewed 24 out of 24 changed files in this pull request and generated 6 comments.

Show a summary per file

File	Description
tests/test_yaml_parser.py	Quote standardization and spacing improvements
tests/test_cli_parser.py	Quote standardization and spacing improvements
tests/test_api_endpoint_config.py	Quote standardization and spacing improvements
src/seclab_taskflow_agent/shell_utils.py	Reformatted function calls and quote standardization
src/seclab_taskflow_agent/render_utils.py	Reformatted function signatures and quote standardization
src/seclab_taskflow_agent/path_utils.py	Reformatted function calls and spacing improvements
src/seclab_taskflow_agent/mcp_utils.py	Extensive reformatting of multi-line function calls and quote standardization
src/seclab_taskflow_agent/mcp_servers/memcache/memcache_backend/sqlite.py	Quote standardization and spacing improvements; contains pre-existing bug
src/seclab_taskflow_agent/mcp_servers/memcache/memcache_backend/sql_models.py	Quote standardization and spacing improvements
src/seclab_taskflow_agent/mcp_servers/memcache/memcache_backend/dictionary_file.py	Quote standardization and spacing improvements
src/seclab_taskflow_agent/mcp_servers/memcache/memcache_backend/backend.py	Trailing whitespace removal
src/seclab_taskflow_agent/mcp_servers/memcache/memcache.py	Quote standardization and spacing improvements
src/seclab_taskflow_agent/mcp_servers/logbook/logbook.py	Quote standardization and spacing improvements
src/seclab_taskflow_agent/mcp_servers/echo/echo.py	Quote standardization and spacing improvements
src/seclab_taskflow_agent/mcp_servers/codeql/mcp_server.py	Extensive reformatting of multi-line function calls
src/seclab_taskflow_agent/mcp_servers/codeql/jsonrpyc/init.py	Protocol method formatting and quote standardization
src/seclab_taskflow_agent/mcp_servers/codeql/client.py	Extensive reformatting of multi-line function calls
src/seclab_taskflow_agent/env_utils.py	Spacing improvements
src/seclab_taskflow_agent/capi.py	Quote standardization and spacing improvements
src/seclab_taskflow_agent/available_tools.py	Quote standardization and spacing improvements
src/seclab_taskflow_agent/agent.py	Import formatting and multi-line function reformatting
src/seclab_taskflow_agent/main.py	Extensive reformatting of imports and function calls
release_tools/publish_docker.py	Reformatted function calls and spacing improvements
release_tools/copy_files.py	Spacing improvements

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-01-26T12:36:01Z

@@ -28,7 +29,7 @@ def set_state(self, key: str, value: Any) -> str:
            session.add(kv)
            session.commit()
        return 'f"Stored value in memory for `{key}`"'


This line contains a string that looks like it should be an f-string but is actually a regular string literal. The outer single quotes prevent this from being evaluated as an f-string. This should be:

return f"Stored value in memory for `{key}`"

Note: This appears to be a pre-existing bug that was not introduced by this formatting PR, but it should be fixed.

Suggested change

return 'f"Stored value in memory for `{key}`"'

return f"Stored value in memory for `{key}`"

Copilot · 2026-01-26T12:36:02Z

-            inputs = task_body.get('inputs', {})
-            prompt = task_body.get('user_prompt', '')
+            name = task_body.get("name", "taskflow")  # placeholder, not used yet
+            description = task_body.get("description", "taskflow")  # placeholder not used yet


Variable description is not used.

Suggested change

description = task_body.get("description", "taskflow") # placeholder not used yet

Copilot · 2026-01-26T12:36:02Z

-            run = task_body.get('run', '')
-            inputs = task_body.get('inputs', {})
-            prompt = task_body.get('user_prompt', '')
+            name = task_body.get("name", "taskflow")  # placeholder, not used yet


This assignment to 'name' is unnecessary as it is redefined before this value is used.

Suggested change

name = task_body.get("name", "taskflow") # placeholder, not used yet

Copilot · 2026-01-26T12:36:02Z

-def parse_prompt_args(available_tools: AvailableTools,
-                      user_prompt: str | None = None):
+
+def parse_prompt_args(available_tools: AvailableTools, user_prompt: str | None = None):


parse_prompt_args returns tuple of size 5 and tuple of size 6.
parse_prompt_args returns tuple of size 5 and tuple of size 6.

Copilot · 2026-01-26T12:36:03Z

+from agents import (
+    Agent,
+    Runner,
+    AgentHooks,
+    RunHooks,
+    result,
+    function_tool,
+    Tool,
+    RunContextWrapper,
+    TContext,
+    OpenAIChatCompletionsModel,
+    set_default_openai_client,
+    set_default_openai_api,
+    set_tracing_disabled,
+)


Import of 'function_tool' is not used.

Copilot · 2026-01-26T12:36:03Z

-from fastmcp import FastMCP, Context # use FastMCP 2.0
+
+# from mcp.server.fastmcp import FastMCP, Context
+from fastmcp import FastMCP, Context  # use FastMCP 2.0


Import of 'Context' is not used.

Suggested change

from fastmcp import FastMCP, Context # use FastMCP 2.0

from fastmcp import FastMCP # use FastMCP 2.0

kevinbackhouse · 2026-01-27T12:34:38Z

Some of the comments here are valid, but they correspond to other linter checks which I want to fix in separate PRs. For example, I'll do a separate PR to remove unused imports.

hatch fmt

e0b8b07

Copilot AI review requested due to automatic review settings January 26, 2026 12:33

Copilot started reviewing on behalf of kevinbackhouse January 26, 2026 12:33 View session

github-advanced-security AI found potential problems Jan 26, 2026

View reviewed changes

Copilot AI reviewed Jan 26, 2026

View reviewed changes

kevinbackhouse marked this pull request as ready for review January 26, 2026 22:06

kevinbackhouse requested review from JarLob, Kwstubbs, m-y-mo, p- and sylwia-budzynska as code owners January 26, 2026 22:06

p- approved these changes Jan 27, 2026

View reviewed changes

kevinbackhouse merged commit 37fc28f into GitHubSecurityLab:main Jan 27, 2026
15 checks passed

kevinbackhouse deleted the fmt branch January 27, 2026 12:44

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

auto-format#137

auto-format#137
kevinbackhouse merged 1 commit intoGitHubSecurityLab:mainfrom
kevinbackhouse:fmt

kevinbackhouse commented Jan 26, 2026

Uh oh!

Check notice

Check notice

Check notice

Check notice

Check notice

Check notice

Copilot AI left a comment

Uh oh!

Copilot AI Jan 26, 2026

Uh oh!

Copilot AI Jan 26, 2026

Uh oh!

Copilot AI Jan 26, 2026

Uh oh!

Copilot AI Jan 26, 2026

Uh oh!

Copilot AI Jan 26, 2026

Uh oh!

Copilot AI Jan 26, 2026

Uh oh!

kevinbackhouse commented Jan 27, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

	return 'f"Stored value in memory for `{key}`"'
	return f"Stored value in memory for `{key}`"

	from fastmcp import FastMCP, Context # use FastMCP 2.0
	from fastmcp import FastMCP # use FastMCP 2.0

Conversation

kevinbackhouse commented Jan 26, 2026

Uh oh!

Check notice

Check notice

Check notice

Check notice

Check notice

Check notice

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Jan 26, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 26, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 26, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 26, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 26, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 26, 2026

Choose a reason for hiding this comment

Uh oh!

kevinbackhouse commented Jan 27, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants